-
Notifications
You must be signed in to change notification settings - Fork 27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Migrations - Capture and replay v0.1.0 - Extracted Comparator and Jupyter #271
Migrations - Capture and replay v0.1.0 - Extracted Comparator and Jupyter #271
Conversation
Signed-off-by: Brian Presley <[email protected]>
…v0.1.0' of github.com:opensearch-project/opensearch-migrations into capture-and-replay-v0.1.0 Signed-off-by: Brian Presley <[email protected]>
Signed-off-by: Brian Presley <[email protected]>
…search-migrations into capture-and-replay-v0.1.0
Signed-off-by: Brian Presley <[email protected]>
Codecov Report
@@ Coverage Diff @@
## capture-and-replay-v0.1.0 #271 +/- ##
===============================================================
- Coverage 61.41% 59.04% -2.38%
+ Complexity 593 584 -9
===============================================================
Files 74 74
Lines 2991 2991
Branches 279 279
===============================================================
- Hits 1837 1766 -71
- Misses 986 1057 +71
Partials 168 168
Flags with carried forward coverage won't be shown. Click here to find out more.
|
You can also remove the kafka-broker and kafka-zookeeper copilot directories as those aren't in use since we are using MSK and didn't integrate yet |
Signed-off-by: Brian Presley <[email protected]>
Signed-off-by: Brian Presley <[email protected]>
Signed-off-by: Brian Presley <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One note about whether this will make our logs messier, but we'll figure that out quickly enough after running it and can remedy quickly.
TrafficCapture/dockerSolution/src/main/docker/docker-compose.yml
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving--despite the broken tests--to streamline getting this merged into the release branch (but we should fix the broken tests)
Signed-off-by: Brian Presley <[email protected]>
Signed-off-by: Brian Presley <[email protected]>
Signed-off-by: Brian Presley <[email protected]>
TrafficCapture/dockerSolution/src/main/docker/docker-compose.yml
Outdated
Show resolved
Hide resolved
...in/java/org/opensearch/migrations/replay/datahandlers/http/HttpJsonTransformingConsumer.java
Outdated
Show resolved
Hide resolved
...in/java/org/opensearch/migrations/replay/datahandlers/http/HttpJsonTransformingConsumer.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Brian Presley <[email protected]>
Signed-off-by: Brian Presley <[email protected]>
…ject/opensearch-migrations into capture-and-replay-v0.1.0
Signed-off-by: Brian Presley <[email protected]>
3d2e3bd
into
opensearch-project:capture-and-replay-v0.1.0
Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Is this a backport? If so, please add backport PR # and/or commits #
Testing
[Please provide details of testing done: unit testing, integration testing and manual testing]
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.